Skip to content

Fix BETAFPVG473_V2 default config #921

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

Fix BETAFPVG473_V2 default config #921

wants to merge 1 commit into from

Conversation

@liyouluo
Copy link
Contributor

@liyouluo liyouluo commented Oct 10, 2025 •
edited by coderabbitai bot
Loading

Hi. Please check.

This time we will modify the default config of the board.

Summary by CodeRabbit

  • New Features

    • Added barometer support (BMP280 and DPS310).
    • Introduced external gyro clock input support.
    • Enabled HD OSD display port over a dedicated UART.

  • Improvements

    • Smarter default receiver setup: automatically configures SBUS or CRSF based on build options.
    • Expanded timer/IO mapping for more flexible peripheral routing.

  • Configuration

    • Updated default board yaw alignment to -45°, improving orientation for compatible frames.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 10, 2025 •
edited
Loading

Walkthrough

Adds new macros and pin mappings in BETAFPVG473_V2 config: enables gyro clock input, barometer support (BMP280/DPS310 over I2C), adjusts UART assignments for MSP display port and SerialRX based on compile-time flags, updates timer pin map with PB9, and sets board yaw alignment to -45.

Changes

Cohort / File(s) Summary
Board config updates
configs/BETAFPVG473_V2/config.h		 Added USE_GYRO_CLKIN with GYRO_1_CLKIN_PIN=PB9 and TIMER_PIN_MAP(5, PB9, 1, -1); enabled barometer macros USE_BARO, USE_BARO_BMP280, USE_BARO_DPS310 with BARO_BUSTYPE=I2C; added MSP_DISPLAYPORT_UART=SERIAL_PORT_UART4 under USE_OSD_HD; conditional SERIALRX_UART selection (USART1 for SBUS, else USART3 with SERIALRX_PROVIDER=CRSF); set DEFAULT_ALIGN_BOARD_YAW=-45.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested reviewers

  • ot0tot
  • haslinghuis
  • blckmn

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description Check ⚠️ Warning The pull request description is too brief and does not follow the required repository template, as it lacks sections on hardware compliance, review requirements, housekeeping guidelines, and the checklist of validations. Please revise the description to use the provided template, include the mandatory hardware compliance statements, note the branch and review requirements, and complete the checklist items to ensure all necessary information is covered.
✅ Passed checks (2 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
Title Check ✅ Passed The title accurately and succinctly describes the primary change of updating the default configuration for the BETAFPVG473_V2 board and directly reflects the content of the changeset.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@liyouluo liyouluo changed the title Fix BETAFPVH725 default config Fix BETAFPVG473_V2 default config Oct 10, 2025
@ot0tot
Copy link
Contributor

ot0tot commented Oct 11, 2025 •
edited
Loading

Is this for the BETAFPVG473_V3?

Closing, as the V2 does not have a baro or gyro clock. The V3 will require a new target as discussed in the Discord.

@ot0tot ot0tot closed this Oct 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@haslinghuis haslinghuis haslinghuis approved these changes

@coderabbitai coderabbitai[bot] coderabbitai[bot] approved these changes

Assignees

@liyouluo liyouluo

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@liyouluo @ot0tot @haslinghuis